Skip to content

Conversation

kostasrim
Copy link
Contributor

Adds docs for acl genpass command

resolves #371

Signed-off-by: kostas <[email protected]>
@kostasrim kostasrim requested a review from Niennienzz August 20, 2025 08:56
@kostasrim kostasrim self-assigned this Aug 20, 2025
Copy link

vercel bot commented Aug 20, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
documentation Ready Ready Preview Comment Aug 22, 2025 5:07am

Comment on lines 23 to 24
- [Simple string reply](https://redis.io/docs/reference/protocol-spec/#simple-strings): `OK` on success.
- [Bulk string reply](https://redis.io/docs/reference/protocol-spec/#bulk-strings): pseudorandom data. By default it contains 64 bytes, representing 256 bits of data. If bits was given, the output string length is the number of specified bits (rounded to the next multiple of 4) divided by 4.
Copy link
Contributor

@vyavdoshenko vyavdoshenko Aug 20, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor

@Niennienzz Niennienzz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I fixed some details (title, redundant simple string reply section, optional bits argument). LGTM!

@kostasrim kostasrim merged commit 2062586 into main Aug 22, 2025
2 checks passed
@kostasrim kostasrim deleted the kpr10 branch August 22, 2025 07:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ACL GENPASS is fully supported
3 participants